Skip to content

Slot Mode changed to select-field. Slot Times changed to time format instead number format#211

Open
criticallimit wants to merge 2 commits into
robertzaage:mainfrom
criticallimit:main
Open

Slot Mode changed to select-field. Slot Times changed to time format instead number format#211
criticallimit wants to merge 2 commits into
robertzaage:mainfrom
criticallimit:main

Conversation

@criticallimit

Copy link
Copy Markdown
Contributor

No description provided.

@criticallimit

criticallimit commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

@robertzaage

Checks failed because of your files in Test Directory which I dont have in my private fork. Please check.
Maybe you can exclude these Test Directories from Checks

@super-sonic-zero

Copy link
Copy Markdown
Contributor

@criticallimit You can create multiple pull requests for different issues if you create a branch for each issue. It is not necessary to use the main branch for pull requests

@criticallimit

Copy link
Copy Markdown
Contributor Author

@super-sonic-zero
Have to get closer to Github - I know ;-)

But exclude directories for the checks can only be done by Robert. And I don´t want to touch and change his Test Code only for passing these checks.

Even when I create a branch for every issue, if the changed code doesn´t match his Test Files the check will fail as long as these testfiles are not excluded. Or do I miss something?

@criticallimit

criticallimit commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

but everyone feel free to install my Test Repo as add-on (now under Apps) in HomeAssistant. Just add a Repository with https://github.com/criticallimit/GroBroTest, install it and copy paste settings. Stopping add on from Robert, start my test add-on and you can go back and forth. Thats the way I do my Tests in Home Assistant.

But remember: its only for my personal testing. Sometimes it´s not working for days because I´m not a coder just a hobbiist who needs dozend of attemps to get something running ;-)

But in the moment it should be working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants